@flocca i think bad idea implement it inside engine.
i think that should be like module. Im sure what many ppl wanna write notification module for this project and we need prepare possibility for that.

Please try refactor your code and make it better.

Ok. I can code this like a zenbot extension, maybe under extensions/notifications/, a place where we can put all the notifications modules (slack.js, email.js, etc.). What do you think about all this @evseevnn?

@flocca Yes! That good idea! You will make a great contribution to the development of this library if you do this.

@flocca
I agree with @evseevnn in this matter. XMPP has already been added to "lib/engine.js". To continue adding messaging options this way only leads to more code pollution. I recommend reading this thread:
#333,
and make note of the comments by @DeviaVir and @tuxitor. Then take a look at the "zentalk" branch on this fork where a more generiic approach is used for messaging: https://github.com/tuxitor/zenbot/tree/zentalk
The documentation should be informative: https://github.com/tuxitor/zenbot/blob/zentalk/docs/Zentalk.md
The "zenxmpp" program should make a good base for implementing any messaging client. Here is an example:

I think I'll close this PR for now, the changes are good, but since you have the intention of rewriting it doesn't make too much sense to keep it around.

Thanks @DeviaVir, I agree.